Skip to content

Conversation

ronjalh
Copy link
Member

@ronjalh ronjalh commented Sep 9, 2026

Testing to see if we can run github pipelines.

@ronjalh ronjalh requested a review from a team as a code owner September 9, 2026 12:37
@ronjalh ronjalh linked an issue Sep 9, 2026 that may be closed by this pull request
@ronjalh ronjalh self-assigned this Sep 9, 2026
@ronjalh ronjalh added US0 Project setup Priority: low labels Sep 9, 2026
Copy link
Member

@anettkva anettkva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix!

One thing though, looks like this is targeting main instead of dev? Since main is behind dev right now, the diff here is pulling in a bunch of commits that are already merged into dev (eslint fix, prettier reformat, .env gitignore, etc.), even though the actual new change is just the one commit moving/updating the workflow file. Might be worth retargeting the base branch to dev so the diff only shows what's actually new here?

Everything else looks solid, tested npm ci + lint + prettier check locally on this branch and it all passes cleanly.

@ronjalh
Copy link
Member Author

ronjalh commented Sep 11, 2026

This should not be merged as there are no runners avaliable so they will never pass.

@ronjalh ronjalh closed this Sep 11, 2026
Sign in to join this conversation on GitHub.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ci: fix GitHub Actions workflow location
2 participants